Skip to content

feat(update): add MergingSnapshotUpdate#682

Open
gty404 wants to merge 5 commits into
apache:mainfrom
gty404:merging-snapshot-update
Open

feat(update): add MergingSnapshotUpdate#682
gty404 wants to merge 5 commits into
apache:mainfrom
gty404:merging-snapshot-update

Conversation

@gty404
Copy link
Copy Markdown
Contributor

@gty404 gty404 commented May 26, 2026

Abstract base for merge-based snapshot operations (MergeAppend, OverwriteFiles, RowDelta, etc.), implementing the filter → write → merge pipeline consistent with Java's MergingSnapshotProducer.

Also fixes SnapshotSummaryBuilder manifest count fields and a use-after-free bug in SnapshotUpdate::DeleteFile.

@gty404 gty404 force-pushed the merging-snapshot-update branch 6 times, most recently from e950bed to 0eebdbf Compare May 26, 2026 12:39
Abstract base for merge-based snapshot operations (MergeAppend,
OverwriteFiles, RowDelta, etc.), implementing the filter → write →
merge pipeline consistent with Java's MergingSnapshotProducer.

Also fixes SnapshotSummaryBuilder manifest count fields and a
use-after-free bug in SnapshotUpdate::DeleteFile.
@gty404 gty404 force-pushed the merging-snapshot-update branch from 0eebdbf to 3085e61 Compare May 26, 2026 15:40
Comment thread src/iceberg/update/merging_snapshot_update.cc Outdated
Comment thread src/iceberg/update/merging_snapshot_update.cc Outdated
Comment thread src/iceberg/update/merging_snapshot_update.cc
Comment thread src/iceberg/update/merging_snapshot_update.cc Outdated
Comment thread src/iceberg/manifest/manifest_filter_manager.cc Outdated
Comment thread src/iceberg/manifest/manifest_merge_manager.cc Outdated
gty404 and others added 4 commits May 27, 2026 14:53
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants